Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sql: remove fake index and statistics collectors pg_catalog tables #61794

Conversation

RichardJCai
Copy link
Contributor

@RichardJCai RichardJCai commented Mar 10, 2021

Release note (sql change): Remove pg_catalog tables that were mistakenly added,
notably all tables that end in _index that are not pg_catalog.pg_classes and all
statistics collector tables that are not pg_stat_activity.

Removing pg statistics collector tables since they are somewhat different from regular pg_catalog tables and it's unlikely for us to implement them. Accessing them will simply error.

pg_metadata_test.go has not yet been updating to reflect that the _index tables should not actually be tables.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@RichardJCai RichardJCai force-pushed the remove_index_and_stat_collector_tables_03102021 branch 3 times, most recently from 3d0108c to 78f16a7 Compare March 10, 2021 21:44
@RichardJCai RichardJCai requested review from mnovelodou, rafiss and a team March 10, 2021 21:45
@RichardJCai RichardJCai marked this pull request as ready for review March 10, 2021 21:45
Copy link
Collaborator

@rafiss rafiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how did you identify the ones to remove?

can we update the diff tool / codegen tool to ignore these unneeded tables?

Copy link
Contributor

@mnovelodou mnovelodou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I created the following issue and assigned it to myself:
#61860

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained

Copy link
Collaborator

@rafiss rafiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm after updating logic tests

Release note (sql change): Remove pg_catalog tables that were mistakenly added,
notably all tables that end in _index that are not pg_catalog.pg_classes and all
statistics collector tables that are not pg_stat_activity.
@RichardJCai RichardJCai force-pushed the remove_index_and_stat_collector_tables_03102021 branch from 78f16a7 to 9fff967 Compare March 11, 2021 20:32
@RichardJCai
Copy link
Contributor Author

TFTR
bors r=rafiss

@craig
Copy link
Contributor

craig bot commented Mar 11, 2021

Build succeeded:

@craig craig bot merged commit f2f37a8 into cockroachdb:master Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants